Skip to content

test(server-nestjs): verrouille la parité d'émission projectMember + garde anti-consommateur fantôme - #2579

Draft
shikanime wants to merge 3 commits into
mainfrom
test/project-member-parity
Draft

test(server-nestjs): verrouille la parité d'émission projectMember + garde anti-consommateur fantôme#2579
shikanime wants to merge 3 commits into
mainfrom
test/project-member-parity

Conversation

@shikanime

Copy link
Copy Markdown
Member

Issues liées

#2574


Quel est le comportement actuel ?

project-members.service.ts émet projectMember.upsert/projectMember.delete via AppEventsService, mais aucun consommateur @OnEvent('projectMember.*') n'existe dans server-nestjs — les événements sont actuellement sans effet.

Quel est le nouveau comportement ?

  • project-members.service.spec.ts : +68 lignes — verrouille l'émission aux bons appels (ajout/retrait de membre) et ajoute une garde négative documentée (référence MIGRATION-PARITY-MATRIX.md) qui échouera si un demi-migrateur ajoute un consommateur sans audit de parité.

Spec verte sur Node 24.

Cette PR introduit-elle un breaking change ?

Non.

Autres informations

Écart latent documenté (pas de régression live) ; à résoudre au moment du cutover project-member.

…e parity

Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr>
Change-Id: Iecda25d80f34897d6c421615f94040536a6a6964
…er guard

Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr>
Change-Id: Ib2a133a080641583d390f3dbbf7fd5fe6a6a6964
Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr>
Change-Id: Ia0bba6241f9518c234ba6424e68e5f7b6a6a6964
@shikanime shikanime closed this Aug 28, 2026
@shikanime
shikanime deleted the test/project-member-parity branch August 28, 2026 15:54
@shikanime shikanime reopened this Aug 28, 2026
@cloud-pi-native-sonarqube

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant